Se puso explicitamente esta versión ya que creo que se usa API a partir de esta linea base. Esto no se cambiara.

Ok, pues por lo menos, pon la versión 6.2.16(Que es la que estamos usando :( , te confirmo que funciona bien)

¿Por que es necesario decrementar la Quota aqui?

Porque si tienes un site que ya usa recursos y instalas el portlet con una cuota por defecto, al inicializarse la cuota del site por defecto ya debe decrementar el uso que ya se esta haciendo de recursos. Sino, te encontrarías que al instalar el portlet cuota con una cuota por defecto, el uso de recursos hasta la fecha no se contabilizaría.

Para eso ya esta la llamada a addQuota con la variable quotaUsed :-).

Has visto que esto no funcionaba bien?

El método addQuota no tiene en cuenta la inicialización por defecto, si se cambia hay que cambiar el método. Por otro lado, entiendo que el método createDefaultQuota esta para eso, como mucho añadirle el parámetro para hacer en un solo paso. Como quieras...

Si que tiene en cuenta la inicializacion por defecto. Como dije antes, para eso esta la variable quotaUsed en dicho método.
Es decir, los dos métodos que has puesto equivalen a la linea que ya existía.
Este cambio no es necesario ya que no cambia nada del comportamiento anterior.

Juan no nos estamos entendiendo. El método addQuota no tiene en cuenta la configuración por defecto que AHORA puedes poner en el fichero portal-ext.properties. Esta modificación es muy necesaria. Mírate la definición de los métodos en la classe QuotaLocalServiceImpl:

public Quota createDefaultQuota(long companyId, long classNameId,
		long classPK) throws SystemException {

	Quota quota = quotaLocalService.fetchQuotaByClassNameIdClassPK(classNameId, classPK);
	
	if (quota ==  null) {
		int quotaAlert = 0;
		long quotaAssigned = 0;
		long quotaUsed = 0;
		int quotaStatus = 0;
		
		if ( PropsUtil.get("lsug.quota.portlet.default.alert")!=null ) if ( !PropsUtil.get("lsug.quota.portlet.default.alert").equals("") ) quotaAlert=Integer.parseInt( PropsUtil.get("lsug.quota.portlet.default.alert") );
		if ( PropsUtil.get("lsug.quota.portlet.default.assigned")!=null ) if ( !PropsUtil.get("lsug.quota.portlet.default.assigned").equals("") ) quotaAssigned=Integer.parseInt( PropsUtil.get("lsug.quota.portlet.default.assigned") );
		if ( PropsUtil.get("lsug.quota.portlet.default.status")!=null ) if ( !PropsUtil.get("lsug.quota.portlet.default.status").equals("") ) quotaStatus=Integer.parseInt( PropsUtil.get("lsug.quota.portlet.default.status") );		

		quota = quotaLocalService.addQuota(companyId, classNameId, classPK, quotaAlert,
			quotaAssigned, quotaUsed, quotaStatus);
	}
	
	return quota;
}

Si quieres lo comentamos por teléfono.

Ok, entendido! Pensare si ese es el mejor sitio.

Gracias.

Hola @acoronadoc. Estoy a punto de acabar el portlet para Liferay 7 y quería mirar lo que puedo añadir de esta pull request. ¿Por qué es necesario añadir el view_file_entry si no influye en la modificación de la cuota, y la nueva excepcion que lanza?

Es necesario para que la cuota se actualice cuando utilizas la funcionalidad de arrastrar ficheros.

¿Te refieres a arrastrar ficheros desde el ordenador a la pagina inicial de la document library?

Si.

Hola de nuevo. He estado haciendo pruebas y en caso de cualquier error al arrastrar un fichero desde una carpeta del sistema operativo, no muestra ninguno (he abierto https://issues.liferay.com/browse/LPS-75051 por ello).

He probado a añadir este cambio y el comportamiento sigue igual (no muestra el mensaje personalizado de cuota por ningún sitio).

¿Puedes describir los pasos a reproducir? He probado tanto Liferay 7 GA4 como 6.2 GA5

Hola Juan, diría que para arreglar esto solo hice este cambio(También hace varios meses de esto y arreglé otras cosas también). No me suena haber tocado nada mas.

Vaya, pues en mi caso no soluciona nada. En tal caso no añadiré este cambio. Por ahora sólo añadiría la mejora de las propiedades por defecto y la traducción al catalán.